feat: add submodule version+data hash info to --version --debug output #87

hwei0 · 2025-12-17T21:58:08Z

Issue #, if available:
#82

Description of changes:

added custom --version output to publish boto3, botocore git tag+commit+data hash (data hash is the hash of the simplified data files). This information will only be exposed if the hidden --debug flag is set; current --version behaviour without the --deubg flag will remain unchanged.

example output

./target/debug/iam-policy-autopilot --version --debug
0.1.2
boto3 version: commit_id=0ca53211df7d2bb2213436a6b19e8cf0b44dee7d, commit_tag=1.42.11, data_hash=8309474F524E2F3DE2F90ADA66F1C5C887358CC503846073663BA311783FF495
botocore version: commit_id=7c770fb210913ff9fa9b35470161c3278786085b, commit_tag=None, data_hash=615E77E2C8C1DD6FD471416D712F4FE5A8E66B45150DC5D4BDC192736A8A42F1

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

iam-policy-autopilot-cli/src/commands.rs

iam-policy-autopilot-policy-generation/build.rs

iam-policy-autopilot-policy-generation/src/api/model.rs

Cargo.toml

iam-policy-autopilot-policy-generation/build.rs

iam-policy-autopilot-policy-generation/src/api/model.rs

iam-policy-autopilot-policy-generation/build.rs

mschlaipfer · 2025-12-18T12:01:35Z

Cargo.toml

 chrono = { version = "0.4", features = ["serde"] }
 uuid = { version = "1.8", features = ["v4"] }
-sha2 = "0.10"
+sha2 = "0.10.9"


I think this is causing the OpenSSL build issue. We use rustls-tls in the reqwest dependency to work around a similar issue. I wonder if using use https://docs.rs/aws-lc-rs/1.15.2/aws_lc_rs/digest/fn.digest.html here would resolve the error?

seems like aws-lc-rs also failed the build for the same reason 💀

i resolved the build failure by adding the following to Cargo.toml depencies:

openssl = { version = "0.10", features = ["vendored"] }

By how much does that increase the binary size?

Not sure if there's a better solution, but if the size increase is significant, we should look into it more.

oh lol the binary just stays the same size:

[current release] hbwei@6c7e67c57386 iam-policy-autopilot-awslabs % du -sh ./target/release/iam-policy-autopilot 53M ./target/release/iam-policy-autopilot [this PR changes] hbwei@6c7e67c57386 iam-policy-autopilot % du -sh target/release/iam-policy-autopilot 52M target/release/iam-policy-autopilot

maybe it's because I just added openssl to the workspace dependencies, which don't get packaged into the binary

mschlaipfer · 2025-12-18T16:21:20Z

The PR title will need a chore: or feat: so it gets merged correctly (I don't think we have the PR title check, which should block merging this, yet).

iam-policy-autopilot-policy-generation/src/embedded_data.rs

…b.com/cross-rs/cross/wiki/Recipes#openssl

iam-policy-autopilot-cli/src/main.rs

iam-policy-autopilot-policy-generation/build.rs

…mments to structs in build.rs

iam-policy-autopilot-policy-generation/build.rs

weibenz1 · 2026-01-06T21:39:42Z

The PR title will need a chore: or feat: so it gets merged correctly (I don't think we have the PR title check, which should block merging this, yet).

don't forget about this

feedback has been addressed, thanks!

iam-policy-autopilot-cli/src/main.rs

iam-policy-autopilot-policy-generation/resources/config/sdks/boto3

iam-policy-autopilot-policy-generation/Cargo.toml

iam-policy-autopilot-policy-generation/build.rs

iam-policy-autopilot-cli/src/commands.rs

… policy-generation; other refactors

iam-policy-autopilot-policy-generation/src/shared_submodule_model.rs

mschlaipfer

Thanks!

hwei0 added 2 commits December 17, 2025 13:46

feat: add version flag and submodule version+data info

912c890

Merge branch 'main' of https://github.com/hwei0/iam-policy-autopilot

9f79162

hwei0 requested review from karanjitsingh, mschlaipfer and weibenz1 December 17, 2025 21:58

weibenz1 reviewed Dec 17, 2025

View reviewed changes

mschlaipfer previously requested changes Dec 18, 2025

View reviewed changes

mschlaipfer reviewed Dec 18, 2025

View reviewed changes

adpaco-aws reviewed Dec 19, 2025

View reviewed changes

iam-policy-autopilot-policy-generation/src/embedded_data.rs Show resolved Hide resolved

hwei0 and others added 5 commits December 19, 2025 14:24

refactor: address comments in PR#87

a6077d4

refactor: fix clippy

869f80a

fix: add openssl, libssl-dev dependenies to pr build workflow

fbf5280

fix: try to resolve build failure with openssl vendored https://githu…

59b4b63

…b.com/cross-rs/cross/wiki/Recipes#openssl

Merge branch 'awslabs:main' into main

6af7ee6

hwei0 changed the title ~~add submodule version+data hash info to --version output~~ add submodule version+data hash info to --version --debug output Jan 5, 2026

weibenz1 reviewed Jan 5, 2026

View reviewed changes

iam-policy-autopilot-cli/src/main.rs Outdated Show resolved Hide resolved

weibenz1 reviewed Jan 5, 2026

View reviewed changes

iam-policy-autopilot-policy-generation/build.rs Show resolved Hide resolved

weibenz1 reviewed Jan 5, 2026

View reviewed changes

iam-policy-autopilot-policy-generation/build.rs Outdated Show resolved Hide resolved

refactor: reset BTreeMap to HashMap in build.rs serde structs, add co…

01b266b

…mments to structs in build.rs

hwei0 requested review from adpaco-aws, mschlaipfer and weibenz1 January 5, 2026 22:58

refactor: reset to btree

c4a739c

weibenz1 previously approved these changes Jan 6, 2026

View reviewed changes

iam-policy-autopilot-policy-generation/build.rs Show resolved Hide resolved

hwei0 changed the title ~~add submodule version+data hash info to --version --debug output~~ feat: add submodule version+data hash info to --version --debug output Jan 6, 2026

hwei0 mentioned this pull request Jan 7, 2026

feat: add boto3 and botocore automated dependency update github worklflows #94

Open

mschlaipfer requested changes Jan 7, 2026

View reviewed changes

refactor: share submodule struct beteeen build.rs and api/model.rs in…

340b7e7

… policy-generation; other refactors

hwei0 dismissed weibenz1’s stale review via 340b7e7 January 7, 2026 22:15

refactor: minor refactor

0b8b719

mschlaipfer requested changes Jan 9, 2026

View reviewed changes

iam-policy-autopilot-policy-generation/src/shared_submodule_model.rs Outdated Show resolved Hide resolved

iam-policy-autopilot-policy-generation/src/shared_submodule_model.rs Outdated Show resolved Hide resolved

docs: update comments and fix typos in shared_submodule_model.rs

ef97c39

weibenz1 approved these changes Jan 12, 2026

View reviewed changes

mschlaipfer approved these changes Jan 12, 2026

View reviewed changes

hwei0 merged commit 3af3cb3 into awslabs:main Jan 12, 2026
12 checks passed

feat: add submodule version+data hash info to --version --debug output #87

feat: add submodule version+data hash info to --version --debug output #87

Uh oh!

Conversation

hwei0 commented Dec 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mschlaipfer Dec 18, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hwei0 Dec 19, 2025

Choose a reason for hiding this comment

Uh oh!

mschlaipfer Jan 7, 2026

Choose a reason for hiding this comment

Uh oh!

mschlaipfer Jan 7, 2026

Choose a reason for hiding this comment

Uh oh!

hwei0 Jan 9, 2026

Choose a reason for hiding this comment

Uh oh!

mschlaipfer commented Dec 18, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

weibenz1 commented Jan 6, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mschlaipfer left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

hwei0 commented Dec 17, 2025 •

edited

Loading

mschlaipfer Dec 18, 2025 •

edited

Loading